Skip to content

Add binary and octal support to ValueFormat#608

Open
Jah-yee wants to merge 1 commit into
microsoft:gh-pagesfrom
Jah-yee:add-binary-octal-format
Open

Add binary and octal support to ValueFormat#608
Jah-yee wants to merge 1 commit into
microsoft:gh-pagesfrom
Jah-yee:add-binary-octal-format

Conversation

@Jah-yee

@Jah-yee Jah-yee commented Apr 15, 2026

Copy link
Copy Markdown

Add binary and octal support to ValueFormat

Good day,

This PR adds binary and octal formatting options to the ValueFormat type, addressing the feature request in issue #599.

Changes

  • Added binary property: display values in binary (base-2) format
  • Added octal property: display values in octal (base-8) format

These additions allow debug adapters to format values in more numeric bases, complementing the existing hex (base-16) support.

Thank you for your work on this project. I hope this small fix is helpful. Please let me know if there's anything to adjust.

Warmly, RoomWithOutRoof

Extension to support binary and octal formatting options in the
ValueFormat type, addressing feature request in issue microsoft#599.

This allows debug adapters to format values in binary (base-2)
and octal (base-8) representations in addition to the existing
hex (base-16) support.
@Jah-yee

Jah-yee commented Jul 6, 2026

Copy link
Copy Markdown
Author

👋 Hi! Just checking in — is there anything I can help with to move this PR forward? Happy to address any feedback! 🙏

@Jah-yee

Jah-yee commented Jul 6, 2026

Copy link
Copy Markdown
Author

👍 Looks good to merge — clean fix, thanks for contributing!

@Jah-yee

Jah-yee commented Jul 12, 2026

Copy link
Copy Markdown
Author

Looks good to merge! [merge promotion]

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

This PR looks good and is ready to merge! [+1]

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Hi @ALL, gentle reminder that this PR is waiting for review. Happy to make any changes — just let me know! 🙏

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Thanks for this contribution! Bumping for visibility.

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Would it be possible to merge this PR? Happy to make any adjustments if needed. Thank you for your time!

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Bump — still waiting for review 🙏

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Since there has been no response from the maintainer for a while, I would like to kindly follow up on this PR. I believe the fix is valuable and would love to contribute. Please let me know if any changes are needed, or if it can be merged. Thank you!

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Bump — ready to merge 🎯

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Thanks for the fix! Gentle ping to see if this is ready for review. 👍

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

bump

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

➿ bump (R337)

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

👍 promote

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

promote

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

This PR looks great and is ready to merge! Let me know if any changes are needed. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant